Skip to content

chore: migrate rust/qrcode to icp-cli#1401

Draft
marc0olo wants to merge 2 commits into
masterfrom
chore/migrate-rust-qrcode-to-icp-cli
Draft

chore: migrate rust/qrcode to icp-cli#1401
marc0olo wants to merge 2 commits into
masterfrom
chore/migrate-rust-qrcode-to-icp-cli

Conversation

@marc0olo

Copy link
Copy Markdown
Member

Summary

  • Replaces dfx.json with icp.yaml using @dfinity/rust@v3.3.0 (backend) and @dfinity/asset-canister@v2.2.1 (frontend)
  • Moves Rust source from src/qrcode_backend/ to backend/ and renames package to backend
  • Rewrites frontend from webpack + dfx declarations to Vite + @icp-sdk/bindgen, updating actor wiring to use safeGetCanisterEnv / ic_env cookie pattern
  • Updates Makefile: 7 numbered tests using icp canister call (6 --query calls + 1 update call), removes dfx/deploy dependency
  • Adds .github/workflows/qrcode.yml CI job for rust-qrcode using ghcr.io/dfinity/icp-dev-env-rust:1.0.0
  • Removes dfx.json, BUILD.md, webpack.config.js, per-example .devcontainer/, and old src/ directory

Test plan

  • icp network start -d && icp deploy && make test in rust/qrcode
  • All 7 Makefile tests pass (6 query + 1 update, covering all options combinations)
  • Frontend builds and renders QR codes in browser

🤖 Generated with Claude Code

@marc0olo marc0olo force-pushed the chore/migrate-rust-qrcode-to-icp-cli branch from f082aa2 to 64f5730 Compare June 17, 2026 17:00
marc0olo and others added 2 commits June 19, 2026 08:32
Replaces dfx.json with icp.yaml using @dfinity/rust@v3.3.0 and
@dfinity/asset-canister@v2.2.1 recipes. Moves Rust source to backend/,
rewrites frontend to Vite + @icp-sdk/bindgen, updates Makefile tests to
use icp canister call, adds CI workflow, and removes dfx artifacts.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@marc0olo marc0olo force-pushed the chore/migrate-rust-qrcode-to-icp-cli branch from 64f5730 to 305d50b Compare June 19, 2026 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant